The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 23, 2026 9:37pm

PR Summary

Low Risk
Low risk UI-only change that tweaks DOM measurement logic for column auto-resize; potential impact is limited to computed column widths and should be caught quickly in manual use.

Overview
Fixes table column auto-resize to better match what users actually see by measuring text within the table container (so inherited styles like letter-spacing apply) and using getBoundingClientRect().width for sub-pixel accuracy.

Auto-resize now formats values based on column type (stringified json, display-formatted date) and skips auto-resize for boolean columns.

^{Reviewed by Cursor Bugbot for commit 5c5b1c5. Configure here.}

Greptile Summary

This PR fixes column auto-resize under-estimation by (1) mounting the hidden measurement span inside the table container so it inherits .workspace-root's letter-spacing: 0.02em, (2) switching from offsetWidth to getBoundingClientRect().width for sub-pixel accuracy, (3) skipping boolean columns, and (4) using the same display transformation per type (JSON.stringify for json, storageToDisplay for date) that the cell renderer uses. The changes are accurate and consistent with how values are displayed in cells (confirmed at lines 2775 and 2781).

Confidence Score: 5/5

Safe to merge — targeted bug fix with no risk of regressions.

No P0/P1 issues found. The measurement logic is consistent with the actual cell renderers (JSON.stringify for json at line 2775, storageToDisplay at line 2781). The overflow:hidden on the container does not affect getBoundingClientRect().width (layout-level API, not affected by paint clipping). All remaining observations are P2 quality notes.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User double-clicks column resize handle] --> B[handleColumnAutoResize called]
    B --> C{column.type === 'boolean'?}
    C -- Yes --> D[Return early — checkboxes have fixed width]
    C -- No --> E[Get host: containerRef.current ?? document.body]
    E --> F[Create hidden span, append to host\ninherits letter-spacing from .workspace-root]
    F --> G[Measure column header width\ngetBoundingClientRect width + 57px]
    G --> H[For each row value]
    H --> I{column.type?}
    I -- json --> J[text = JSON.stringify val]
    I -- date --> K[text = storageToDisplay val]
    I -- other --> L[text = String val]
    J & K & L --> M[Measure text width\ngetBoundingClientRect width + 17px]
    M --> N[Track maxWidth]
    N --> O{More rows?}
    O -- Yes --> H
    O -- No --> P[newWidth = Math.min ceil maxWidth, 600]
    P --> Q[setColumnWidths + persist metadata]
    F -->|finally| R[host.removeChild measure span]

_{Reviews (1): Last reviewed commit: "fix(tables): account for letter-spacing ..." | Re-trigger Greptile}